-
Notifications
You must be signed in to change notification settings - Fork 111
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: some typos on the terms page #5468
Conversation
Deployment failed with the following error:
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Skipped Deployments
|
WalkthroughThe changes update the terms and conditions text for a retroactive COW token reward program. Modifications include correcting capitalization at the beginning of sentences, replacing semicolons with periods, and ensuring consistent punctuation throughout the document. These adjustments enhance grammatical consistency without altering the substantive content or meaning of the terms. No changes were made to exported or public declarations. Changes
Poem
📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
⏰ Context from checks skipped due to timeout of 90000ms (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
<li>be minimum 18 years of age.</li> | ||
<li>Not be an employee, affiliate, or immediate family member of the Organizer.</li> | ||
<li>be minimum 18 years of age;</li> | ||
<li>not be an employee, affiliate, or immediate family member of the Organizer.</li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wouldn't you want to do the opposit, start each bullet point with capital letter?
For example, in line 761 a bullet list used capital letters
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@@ -711,8 +711,8 @@ export default function Page() { | |||
Must be a real user who conducted trades on CoW Swap on the Base Network during February 2025 and must: | |||
</p> | |||
<ul> | |||
<li>be minimum 18 years of age.</li> | |||
<li>Not be an employee, affiliate, or immediate family member of the Organizer.</li> | |||
<li>be minimum 18 years of age;</li> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why replacing the dot with a semicolon?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Summary
Fixes some typos on the terms page reported in #5460 (review)


Summary by CodeRabbit
Summary by CodeRabbit